-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[chore] bump dependencies #7520
Conversation
Asset Size Report for c673682 IE11 Builds 🛑 The size of the library EmberData has increased by +542.0 B (+163.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (IE11)
Modern Builds 🛑 The size of the library EmberData has increased by +611.0 B (+174.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) 🛑 The size of the library EmberData has increased by +1.26 KB (+202.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (Modern)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good if CI works
f15fd78
to
a6af60b
Compare
Performance Report for c673682 Scenario - materialization:
|
d38ef85
to
aec99a8
Compare
@@ -22,7 +22,8 @@ import Store, { | |||
import { identifierCacheFor } from '@ember-data/store/-private'; | |||
|
|||
type StableRecordIdentifier = import('@ember-data/store/-private/ts-interfaces/identifier').StableRecordIdentifier; | |||
type ExistingResourceObject = import('@ember-data/store/-private/ts-interfaces/ember-data-json-api').ExistingResourceObject; | |||
type ExistingResourceObject = | |||
import('@ember-data/store/-private/ts-interfaces/ember-data-json-api').ExistingResourceObject; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in-range updates to lint and prettier enforce the max-line-length for these type declarations now (they didn't before) which is why some of them now split lines.
tag: Tag; | ||
owner: any; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was typed as any
already, and there is no import for it yet in ember-qunit 5 (ember-test-helpers is not available in the ember-qunit 5 world)
@@ -889,53 +887,6 @@ module('integration/relationship/belongs_to Belongs-To Relationships', function | |||
assert.ok(comment instanceof Message, 'a comment is an instance of a message'); | |||
}); | |||
|
|||
test('relationshipsByName does not cache a factory', async function (assert) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this test covered a case where in pre 1.x versions of ember-data we stashed the factory onto relationship meta directly as the type
field. It has not been the case that we do this since. The type
field today is a string and other tests would fail were it not. This test no longer has value, so I removed it.
That said, it also failed because the this
context of the test is registered as a destroyable by setupTest
and thus cannot be torndown and then reused. We don't have any other tests like this but if we ever add one the solution is to not use setupTest
at all for this test and to instead setupContext/teardownContext with objects we control.
// Handle testing feature flags | ||
if (QUnit.urlParams.enableoptionalfeatures) { | ||
window.EmberDataENV = { ENABLE_OPTIONAL_FEATURES: true }; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this used to be in the tests/index.html file but QUnit is no longer a global.
@@ -10,6 +10,8 @@ module.exports = Object.assign({}, addonBaseConfig, { | |||
shouldRollupPrivate: true, | |||
externalDependenciesForPrivateModule() { | |||
return [ | |||
'ember-cached-decorator-polyfill', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the polyfill rewrites the import from @glimmer/tracking
to be from itself so we declare it here now.
This PR
node: 'current'
to targets for fastboot builds.In the process we pick up a drift of prettier that causes the bulk of the line changes here due to a change in the formatting of some code.